Skip to content

Ruby refresher done#2

Open
fabienalbi wants to merge 11 commits intoParisRubyWorkshop:masterfrom
fabienalbi:master
Open

Ruby refresher done#2
fabienalbi wants to merge 11 commits intoParisRubyWorkshop:masterfrom
fabienalbi:master

Conversation

@fabienalbi
Copy link

No description provided.

lib/questions.rb Outdated

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Space missing to the left of {.

lib/questions.rb Outdated

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Redundant return detected.

lib/questions.rb Outdated

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Line is too long. [184/80]

lib/questions.rb Outdated

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Line is too long. [145/80]

lib/questions.rb Outdated

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use %w or %W for an array of words.

lib/questions.rb Outdated

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Move puts string out of the conditional.

lib/questions.rb Outdated

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Align the elements of an array literal if they span more than one line.

lib/questions.rb Outdated

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indent the right bracket the same as the start of the line where the left bracket is.

lib/questions.rb Outdated

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unnecessary spacing detected.
Operator = should be surrounded by a single space.

@bu7ch
Copy link
Member

bu7ch commented Apr 9, 2016

au moins c'est utile à produire du code plus performant...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments